Remove broken optimization in style modifier overrides
authorAlexander Larsson <alexl@redhat.com>
Tue, 6 Dec 2011 13:09:45 +0000 (14:09 +0100)
committerAlexander Larsson <alexl@redhat.com>
Tue, 6 Dec 2011 13:12:23 +0000 (14:12 +0100)
We used to only set the override color or font if it was different
than the existing value. However, that means you can't change it to
an override that is the same as the default value for the property.

With this fixed you can e.g. override with a color of 0,0,0,0 which
you couldn't before.

gtk/gtkmodifierstyle.c

index 8d4a18cb89108fc317f30903bf993935a411f4a6..fd84cae0f4b63adc9e6df8f9e6de767115638674 100644 (file)
@@ -162,16 +162,6 @@ modifier_style_set_color (GtkModifierStyle *style,
   g_return_if_fail (GTK_IS_MODIFIER_STYLE (style));
 
   priv = style->priv;
-  gtk_style_properties_get (priv->style, state,
-                            prop, &old_color,
-                            NULL);
-
-  if ((!color && !old_color) ||
-      (color && old_color && gdk_rgba_equal (color, old_color)))
-    {
-      gdk_rgba_free (old_color);
-      return;
-    }
 
   if (color)
     gtk_style_properties_set (priv->style, state,
@@ -214,19 +204,6 @@ _gtk_modifier_style_set_font (GtkModifierStyle           *style,
   g_return_if_fail (GTK_IS_MODIFIER_STYLE (style));
 
   priv = style->priv;
-  gtk_style_properties_get (priv->style, 0,
-                            "font", &old_font,
-                            NULL);
-
-  if ((!old_font && !font_desc) ||
-      (old_font && font_desc &&
-       pango_font_description_equal (old_font, font_desc)))
-    {
-      if (old_font)
-        pango_font_description_free (old_font);
-
-      return;
-    }
 
   if (font_desc)
     gtk_style_properties_set (priv->style, 0,